Skip to content

[DO NOT MERGE] August 24th Release #2249

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 28 commits into from
Aug 23, 2017
Merged

[DO NOT MERGE] August 24th Release #2249

merged 28 commits into from
Aug 23, 2017

Conversation

josenavas
Copy link
Contributor

No description provided.

antgonza and others added 28 commits August 16, 2017 07:41
@antgonza
fix #2229
79b232f
@antgonza
cleaning html
2d48a65
@antgonza
fix #2225 and partial #2224
ee0deb4
@antgonza
fix errors
4dc760b
@antgonza
fix #2224 and fix #2223
868a9e0
@antgonza
fixing comments/errors
e5ff2c8
@josenavas @antgonza
Remove race condition - Fixes 2143 (#2203)
24b7d8e 
* Removing race condition

* Submit the validator release as another job

* Solving some tests

* Fixing test

* Addressing @antgonza's comments
@antgonza
fixing error
51ff4f9
@antgonza
fix #2216
adf98a3
@adswafford
Merge pull request #2243 from antgonza/fix-2216
4af4656 
fix #2216
@adswafford
Merge pull request #2242 from antgonza/fix-2225
a450aeb 
fix #2225 and partial #2224
@adswafford
Merge pull request #2241 from antgonza/fix-2229
db0a4b6 
fix #2229
@antgonza
delete-root-analysis-artifact
899268a
@antgonza
fix errors
3e17876
@josenavas
Merge pull request #2244 from antgonza/delete-root-analysis-artifact
a369cad 
delete-root-analysis-artifact
@antgonza
fix #2210
773c8da
@antgonza
fix error
4bf4e97
@antgonza
fixing test
a5836b1
@antgonza
fix errors
c925756
@josenavas
fixes 1880
19b4fe3
@josenavas
Removing commented code
d1bbe46
@josenavas
Fixing test
63ccfd9
@antgonza @ElDeveloper
fix #2211 (#2240)
acd27f5 
* fix #2211

* following @qiita-dev suggestions

* fixing errors and improving message

* removing Ó

* 🐾

* æ
@josenavas
DEBUG: adding prints to see the actual error
eae20cc
@josenavas
Fixing test and remove debug code
b7e2cc9
@tanaes
Merge pull request #2247 from josenavas/fix-1880
39b2b84 
Fixes 1880
@tanaes
Merge pull request #2246 from antgonza/fix-2210
f94af55 
Fix 2210
@josenavas
Merge pull request #2248 from biocore/dev
7941067 
August 23rd Release Candidate
@codecov-io
Copy link

codecov-io commented Aug 21, 2017 •
edited
Loading

Codecov Report

Merging #2249 into master will decrease coverage by 0.02%.
The diff coverage is 91.58%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2249      +/-   ##
==========================================
- Coverage   92.84%   92.81%   -0.03%     
==========================================
  Files         171      171              
  Lines       18877    18925      +48     
==========================================
+ Hits        17527    17566      +39     
- Misses       1350     1359       +9
Impacted Files Coverage Δ
...ta_pet/handlers/artifact_handlers/base_handlers.py 93.25% <ø> (ø) ⬆️
...ta_db/metadata_template/test/test_prep_template.py 99.81% <ø> (-0.01%) ⬇️
qiita_pet/handlers/api_proxy/tests/test_studies.py 99.43% <ø> (ø) ⬆️
qiita_pet/handlers/study_handlers/prep_template.py 57.4% <ø> (ø) ⬆️
...lers/artifact_handlers/tests/test_base_handlers.py 95.34% <ø> (ø) ⬆️
qiita_db/metadata_template/util.py 90.97% <100%> (-0.07%) ⬇️
qiita_db/test/test_util.py 99.78% <100%> (ø) ⬆️
...iita_pet/handlers/api_proxy/tests/test_artifact.py 98.22% <100%> (+0.03%) ⬆️
qiita_db/util.py 96.55% <100%> (+0.03%) ⬆️
qiita_db/test/test_processing_job.py 99.8% <100%> (ø) ⬆️
... and 9 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6b59f37...7941067. Read the comment docs.

@josenavas josenavas merged commit 6080742 into master Aug 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@josenavas @codecov-io @antgonza @adswafford @tanaes